New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
8234704: Fix attribution in libxslt.md #52
Conversation
👋 Welcome back kcr! A progress list of the required criteria for merging this PR into |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"THE DANIEL VEILLARD"
The one and only ? Was THE supposed to be here ?
Strange as it may seem, yes. This is taken exactly from the |
@kevinrushforth This change can now be integrated. The commit message will be:
|
/integrate |
@kevinrushforth |
Mailing list message from Kevin Rushforth on openjfx-dev: Changeset: 126896d 8234704: Fix attribution in libxslt.md Reviewed-by: prr ! modules/javafx.web/src/main/legal/libxslt.md |
We discovered three minor differences between the
libxslt.md
file and the libxslt-1.1.34 source bundle:The copyright years didn't match the source
The following line was missing:
BLFS
rather thanDANIEL VEILLARD
in one place.Note to reviewers:
I pushed this in two commits as an aid to reviewing it (as with any other PR, it will be squashed during integration). The first commit just fixes the above three issues without any reformatting of the lines. The second commit has no changes in the content itself, but re-wraps the lines to match the source.
Progress
Issue
JDK-8234704: Fix attribution in libxslt.md
Approvers